Skip to content

🗃️ Structure import#2504

Draft
AndyScherzinger wants to merge 15 commits intomainfrom
feat/noid/structureImport
Draft

🗃️ Structure import#2504
AndyScherzinger wants to merge 15 commits intomainfrom
feat/noid/structureImport

Conversation

@AndyScherzinger
Copy link
Copy Markdown
Member

WIP for CI execution/checks

@AndyScherzinger AndyScherzinger added enhancement New feature or request 2. developing Work in progress labels Apr 20, 2026
@AndyScherzinger AndyScherzinger force-pushed the feat/noid/structureImport branch 6 times, most recently from 4740f30 to bca53a5 Compare April 21, 2026 18:28
@AndyScherzinger AndyScherzinger modified the milestone: v2.1.0 Apr 24, 2026
…rison

AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
…ges atomically

AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
…er works

The <input> was inside <template #actions> which gets unmounted when the
menu closes. Moving it to the default slot keeps it in the DOM so
.importStructureFileInput.click() can open the file picker.

AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
…rted {id} template

generateOcsUrl() does not support {id} placeholder substitution — it just
prepends the OCS base path. The literal string '{id}' was being included in
the URL, causing 404s silently swallowed by the catch blocks.

AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
…tations

StructureDiffService was returning a flat structure that ImportStructurePreview
was unable to render:

- tableMeta: was {changes: [{field, current, incoming}]}, now {field: {current, incoming}}
- columns add: was {action, sourceId, title, type, details}, now {action, column: <srcCol>}
- columns update: was {action, sourceId, targetId, title, type, changes: [{field,...}]},
  now {action, targetId, column: {title, type}, changes: {field: {current, incoming}}}
- columns delete: was {action, targetId, title, type},
  now {action, targetId, column: {title, type}}
- views add: was {action, title, details}, now {action, view: <srcView>}
- views update changes: was [{field, current, incoming}], now {field: {current, incoming}}

Updated StructureDiffServiceTest assertions to match the corrected shapes.
Also removed the :disabled guard that prevented Preview from firing without
a selected file — replaced with a showWarning call inside the handler.

AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
…ogic

AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
AI-assistant: Copilot 1.0.6 (Claude Sonnet 4.6)
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
@AndyScherzinger AndyScherzinger force-pushed the feat/noid/structureImport branch from bca53a5 to d6ec125 Compare April 25, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant